Skip to content

Fix test reporting from installer builds#786

Closed
papeh wants to merge 8 commits intomainfrom
fix/tests4CD
Closed

Fix test reporting from installer builds#786
papeh wants to merge 8 commits intomainfrom
fix/tests4CD

Conversation

@papeh
Copy link
Copy Markdown
Contributor

@papeh papeh commented Mar 25, 2026

  • Separate test and install builds
    Failing tests were not failing installer builds. Now they do.
  • Fix garbled comment
  • Ignore test residue
  • Remove redundant -BuildTests flag (it is implied by -RunTests)

This change is Reviewable

@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 25, 2026

NUnit Tests

    1 files  ±0      1 suites  ±0   5m 55s ⏱️ -2s
4 074 tests ±0  4 003 ✅ ±0  71 💤 ±0  0 ❌ ±0 
4 083 runs  ±0  4 012 ✅ ±0  71 💤 ±0  0 ❌ ±0 

Results for commit d35c964. ± Comparison against base commit c450a55.

♻️ This comment has been updated with latest results.

papeh added 5 commits March 25, 2026 14:30
Failing tests were not failing installer builds. Now they do.
This reverts commit 68b3d01.
That commit didn't fix CD builds.
@jasonleenaylor
Copy link
Copy Markdown
Contributor

I think this will make the build run 3 times.

@papeh
Copy link
Copy Markdown
Contributor Author

papeh commented Mar 25, 2026

Then I will try again after #784 is merged.

@papeh papeh closed this Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants